Skip to content

Conversation

cujomalainey
Copy link
Contributor

This is a re-attempt of #77492 minus the API modification patches that will be modified and sent in a formal change later outside of the driver specific changes.

@github-actions github-actions bot added the area: SPI SPI bus label Mar 26, 2025
@github-actions github-actions bot requested review from tbursztyka and teburd March 26, 2025 22:57
@carlescufi carlescufi requested a review from anangl March 27, 2025 11:12
@tbursztyka
Copy link
Contributor

Rename the PR, it's improving driver support but not implementing enhanced SPI operation.

I think you could drop the first commit about rx-delay, see #87427 (if you have input on such improvements, please do)

@cujomalainey cujomalainey changed the title Support enhanced SPI operation in spi_dw driver Enhancements for spi_dw driver Mar 27, 2025
@cujomalainey
Copy link
Contributor Author

cujomalainey commented Mar 27, 2025

Rename the PR, it's improving driver support but not implementing enhanced SPI operation.

I think you could drop the first commit about rx-delay, see #87427 (if you have input on such improvements, please do)

yep that indeed appear to be the same api, i will drop that patch and review it later for compatibility, but a surface level check appears to be exactly what we need.

The DFS(Data Frame Size) is at CTRLR0[4:0] in HSSI controller.

Signed-off-by: Younghyun Park <[email protected]>
The TXFTLR register has 2 major fields which are TFT for triggering
interrupt threshold and TXFTLR for starting transfer threshold. This is
to ensure that sufficient data is ready for starting transfer.

Signed-off-by: Younghyun Park <[email protected]>
Read the Synopsys SSI component version to extend supported capability
based on the version.

Signed-off-by: Younghyun Park <[email protected]>
@cujomalainey
Copy link
Contributor Author

@tbursztyka mind if i get a re-review?

@cujomalainey
Copy link
Contributor Author

@teburd @anangl mind reviewing so I can get this in?

@kartben kartben merged commit db16853 into zephyrproject-rtos:main Apr 25, 2025
25 checks passed
@cujomalainey cujomalainey deleted the dw_spi branch April 26, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: SPI SPI bus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants